server/src/main/java/com/linecorp/centraldogma/server/internal/api/auth/ApplicationCertificateAuthorizer.java (1)

77-79: Consider showing class names in the error message for easier debugging.

The current message will print object references. Showing class names would help operators identify which implementations are conflicting.

♻️ Optional improvement for clearer error message

         } else {
             throw new IllegalStateException(
                     "Only one ApplicationCertificateIdExtractor implementation must be provided. " +
-                    "found: " + extractors);
+                    "found: " + extractors.stream()
+                                          .map(e -> e.getClass().getName())
+                                          .collect(Collectors.toList()));
         }

Add the import:

import java.util.stream.Collectors;

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@server/src/main/java/com/linecorp/centraldogma/server/internal/api/auth/ApplicationCertificateAuthorizer.java`
around lines 77 - 79, The IllegalStateException in
ApplicationCertificateAuthorizer currently prints the extractors list (object
references); update the throw to include the concrete class names of the
conflicting ApplicationCertificateIdExtractor implementations by mapping
extractors to their getClass().getName() (or similar) and joining them (e.g.,
using Collectors.joining) so the exception message shows the implementation
class names rather than raw object toString() values; keep the same exception
text but replace "found: " + extractors with the joined class-name string for
easier debugging.